-
Notifications
You must be signed in to change notification settings - Fork 753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Init profiles worker with member affiliations update #1962
Merged
loicsaintroch
merged 17 commits into
main
from
improvement/async-member-affiliations-update
Jan 8, 2024
Merged
Init profiles worker with member affiliations update #1962
loicsaintroch
merged 17 commits into
main
from
improvement/async-member-affiliations-update
Jan 8, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
…ber-affiliations-update
appy
…ber-affiliations-update
sausage-todd
pushed a commit
that referenced
this pull request
Feb 22, 2024
sausage-todd
pushed a commit
that referenced
this pull request
Feb 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes proposed ✍️
What
🤖[deprecated] Generated by Copilot at 80fba34
This pull request introduces a new profiles-worker service that uses Temporal workflows to update member affiliations in the database. It adds a new directory
services/apps/profiles_worker
that contains the service code, configuration, and types. It also adds a new Dockerfile, a Docker Compose configuration, and a shell script for building and running the service. It modifies the existing GitHub workflows and thememberRepository.ts
file to use the profiles-worker service instead of the old logic. It updates thepnpm-lock.yaml
file to reflect the new dependencies for the service.
🤖[deprecated] Generated by Copilot at 80fba34
Why
How
🤖[deprecated] Generated by Copilot at 80fba34
MemberAffiliationRepository.update
in the backend service with calls tooptions.temporal.workflow.start
that start thememberUpdate
workflow defined by the profiles-worker service (link, link, link)MemberAffiliationRepository
from the backend service and add the import ofTemporalWorkflowId
enum from the@crowd/types
package (link, link, link)deploy_profiles_worker
to the GitHub workflows for deploying to the LF production, production, and staging environments that allows the user to choose whether to deploy the profiles-worker service or not (link, link)build-and-push-profiles-worker
anddeploy-profiles-worker
to the GitHub workflows for deploying to the LF production, production, and staging environments that build and push the Docker image for the profiles-worker service and deploy the service to the respective clusters using AWS EKS and Helm (link, link, link, link)lf-staging/**
,lf-staging-**
,staging/**
, orstaging-**
branches that affect the profiles-worker service or its dependencies (link, link)uuid
package from8.3.0
to8.3.2
and remove the unused resolution for the[email protected]
package in thepnpm-lock.yaml
file (link, link)Checklist ✅
Feature
,Improvement
, orBug
.